Skip to content

chore(test): enforce Vitest coverage thresholds in CI#797

Merged
RUKAYAT-CODER merged 2 commits into
rinafcode:mainfrom
Olasunkanmi975:chore/coverage-threshold-enforcement
Jun 29, 2026
Merged

chore(test): enforce Vitest coverage thresholds in CI#797
RUKAYAT-CODER merged 2 commits into
rinafcode:mainfrom
Olasunkanmi975:chore/coverage-threshold-enforcement

Conversation

@Olasunkanmi975

Copy link
Copy Markdown
Contributor

##closed #751

Summary

Adds automated coverage enforcement to the test pipeline by configuring minimum Vitest coverage thresholds, integrating coverage reporting into CI, and documenting the project's coverage policy.

Changes

  • Configured Vitest to use the V8 coverage provider with minimum thresholds:

    • Lines: 60%
    • Statements: 60%
    • Functions: 60%
    • Branches: 50%
  • Updated the CI workflow to run coverage with JSON and LCOV reporters.

  • Added LCOV upload for coverage trend tracking (using the repository's configured coverage service).

  • Documented the coverage policy and local coverage workflow in CONTRIBUTING.md.

Testing

Verified:

  • Coverage reports are generated in JSON and LCOV formats.
  • Coverage threshold failures produce a non-zero exit code.
  • CI reports coverage as part of the pull request checks.

Validation

npm test
npm run test:coverage
npm run lint

Notes

This change introduces coverage enforcement only. It does not modify application behavior or existing test logic.

@drips-wave

drips-wave Bot commented Jun 27, 2026

Copy link
Copy Markdown

@Olasunkanmi975 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@Olasunkanmi975

Copy link
Copy Markdown
Contributor Author

Hi maintainer, please review this pr

@RUKAYAT-CODER

Copy link
Copy Markdown
Contributor

Great job so far

There’s just one blocker — the workflow is failing. Could you take a look and fix it so all checks pass?

Happy to review again once that’s done.
You can pull from the main first before pushing. The workflow should pass

@RUKAYAT-CODER

Copy link
Copy Markdown
Contributor

Workflow is still failing

@Olasunkanmi975

Copy link
Copy Markdown
Contributor Author

@RUKAYAT-CODER2025 please review there is nothing to fix there all good

@RUKAYAT-CODER

Copy link
Copy Markdown
Contributor

Workflow is still failing

@Olasunkanmi975 Olasunkanmi975 force-pushed the chore/coverage-threshold-enforcement branch from b9a48c7 to c95bc76 Compare June 29, 2026 11:37
@Olasunkanmi975

Copy link
Copy Markdown
Contributor Author

@RUKAYAT-CODER Hi maintainer please review this pr and merge, all resolved

@Olasunkanmi975

Copy link
Copy Markdown
Contributor Author

@RUKAYAT-CODER Hi maintainer please review this pr and merge, all resolved

@RUKAYAT-CODER

Copy link
Copy Markdown
Contributor

Thank you for contributing to the project.

@RUKAYAT-CODER RUKAYAT-CODER merged commit e087786 into rinafcode:main Jun 29, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Tech-debt] No code coverage thresholds configured in vitest.config.ts

2 participants